Skip to content

Feat/64 65 66/columns cards comments api#98

Merged
ToKyun02 merged 9 commits intomainfrom
feat/64-65-66/columns-cards-comments-API
Feb 11, 2025
Merged

Feat/64 65 66/columns cards comments api#98
ToKyun02 merged 9 commits intomainfrom
feat/64-65-66/columns-cards-comments-API

Conversation

@ToKyun02
Copy link
Owner

@ToKyun02 ToKyun02 commented Feb 11, 2025 •
edited
Loading

β“μ΄μŠˆ

✍️ Description

/columns, /cards, /comments API ν•¨μˆ˜ μž‘μ—…μ— κ΄€ν•œ PRμž…λ‹ˆλ‹€.

μž‘μ—… λ‚΄μš©μ€ λ‹€μŒκ³Ό κ°™μŠ΅λ‹ˆλ‹€.

  1. 각 도메인 별 μŠ€ν‚€λ§ˆ 및 νƒ€μž… μž‘μ„±
  2. 각 도메인 별 API ν•¨μˆ˜ μž‘μ„±

URL κ²½λ‘œμ— idλ₯Ό μ„€μ •ν•΄μ•Ό ν•˜λŠ” κ²½μš°μ—λŠ” form μŠ€ν‚€λ§ˆμ—μ„œ μ œμ™Έν–ˆκ³ , api ν•¨μˆ˜μ—μ„œ λ”°λ‘œ idλ₯Ό 받도둝 νŒŒλΌλ―Έν„°λ₯Ό μ„€μ •ν–ˆμŠ΅λ‹ˆλ‹€.

form으둜 λ°›μ•„μ•Όν•˜λŠ” idλŠ” input readonly, hidden μ²˜λ¦¬ν•΄μ„œ form λ°μ΄ν„°λ‘œ λ„˜κ²¨μ£Όμ‹œλ©΄ 될 것 κ°™μŠ΅λ‹ˆλ‹€.

date validation λ•Œλ¬Έμ— dayjs λͺ¨λ“ˆ μΆ”κ°€ μ„€μΉ˜ν–ˆμŠ΅λ‹ˆλ‹€.

βœ… Checklist

PR

  • Branch Convention 확인

    feat/* κΈ°λŠ₯ κ΅¬ν˜„, fix/* 버그 μˆ˜μ •, refactor/* κ°œμ„ 

  • Base Branch 확인
  • μ μ ˆν•œ Label 지정
  • Assignee 및 Reviewer 지정

Test

  • 둜컬 μž‘λ™ 확인

Additional Notes

  • deleteCard() response return νƒ€μž… μˆ˜μ •

@ToKyun02 ToKyun02 added Priority: Medium πŸ”– μš°μ„ μˆœμœ„ 보톡 Type: Feature ✨ μ‹ κ·œ κΈ°λŠ₯ labels Feb 11, 2025
@ToKyun02 ToKyun02 requested a review from cksrlcks February 11, 2025 02:26
@ToKyun02 ToKyun02 self-assigned this Feb 11, 2025
@vercel
Copy link

vercel bot commented Feb 11, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
taskify βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Feb 11, 2025 2:34am

cksrlcks
cksrlcks reviewed Feb 11, 2025
View reviewed changes
cksrlcks
cksrlcks reviewed Feb 11, 2025
View reviewed changes
cksrlcks
cksrlcks approved these changes Feb 11, 2025
View reviewed changes
Copy link
Collaborator

@cksrlcks cksrlcks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μ—­μ‹œ μ—„μ²­ κΉ”λ”ν•˜κ²Œ μž˜μ§œμ‹œλ„€μš₯γ…Ž
λ¦¬μ•‘νŠΈμΏΌλ¦¬μ—μ„œ νŒ¨μΉ­ν•¨μˆ˜λ‘œλ„ λ°”λ‘œ μ‚¬μš©κ°€λŠ₯ν•˜κ²Œ ν•΄μ£Όμ…”μ„œ νŽΈλ¦¬ν• κ²ƒκ°™μ•„μš”

@ToKyun02 ToKyun02 merged commit 4e8e6b8 into main Feb 11, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cksrlcks cksrlcks cksrlcks approved these changes

Assignees

@ToKyun02 ToKyun02

Labels

Priority: Medium πŸ”– μš°μ„ μˆœμœ„ 보톡 Type: Feature ✨ μ‹ κ·œ κΈ°λŠ₯

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feat: /cards API ν•¨μˆ˜ κ΅¬ν˜„ Feat: /columns API ν•¨μˆ˜ κ΅¬ν˜„ Feat: /comments API ν•¨μˆ˜ κ΅¬ν˜„

2 participants

@ToKyun02 @cksrlcks